Skip to content

Add CrystalToolkitAnimationScene to phonon component#476

Merged
tschaume merged 18 commits intomaterialsproject:mainfrom
minhsueh:add_animation
Oct 24, 2025
Merged

Add CrystalToolkitAnimationScene to phonon component#476
tschaume merged 18 commits intomaterialsproject:mainfrom
minhsueh:add_animation

Conversation

@minhsueh
Copy link
Copy Markdown
Collaborator

@minhsueh minhsueh commented Sep 9, 2025

Major changes:

  • Add CrystalToolkitAnimationScene to the phonon component
  • Implemented a callback that updates the animation based on the selected q-point and band in the dispersion plot

@minhsueh minhsueh marked this pull request as draft September 9, 2025 23:53
@minhsueh minhsueh marked this pull request as ready for review September 12, 2025 23:11
Comment thread crystal_toolkit/components/phonon.py Outdated
Comment thread crystal_toolkit/components/phonon.py Outdated
@tschaume
Copy link
Copy Markdown
Member

@minhsueh I took a pass at the code. Please review and test. Thank you!

@minhsueh
Copy link
Copy Markdown
Collaborator Author

@tschaume I've cleaned up the code, renamed several variables for clarity, and added explanations about how the Scene works. These updates will be helpful for future maintenance. :)

@tschaume tschaume merged commit 548c6fa into materialsproject:main Oct 24, 2025
6 checks passed
tschaume added a commit that referenced this pull request Nov 4, 2025
@mkhorton
Copy link
Copy Markdown
Member

Thank you for adding this @minhsueh!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants